Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

anthro osteology KS changes #62

Merged
merged 1 commit into from
Sep 11, 2020
Merged

Conversation

kspurgin
Copy link
Collaborator

No description provided.

Copy link
Collaborator Author

@kspurgin kspurgin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kevin-kariuki Please review changes and merge into #57 . Thanks!

@@ -34,12 +34,12 @@
"cranialDeformationCategory": "other (describe);tabular;circumferential",
"cranialDeformationNote": "of cranial deformation present",
"trepanationPresent": "false",
"trepanationLocation": "bone and side^^^^Gang Gang",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no repeating field subgroup inside this repeating field group, so use ; delimiter

"trepanationDimensionMax": "50;;90",
"trepanationDimensionMin": ";60;100",
"trepanationTechnique": "grooving;;grooving",
"trepanationHealing": "possible healing;no healing;definite evidence for healing",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no healing instead of healing

@@ -8,7 +8,7 @@
"profile_basename": "anthro",
"ns_uri": {
"osteology_common": "http://collectionspace.org/services/osteology",
"osteology_anthropology": null
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was causing the major initial error. I manually fixed the RecordMapper for osteology in the mapper fixtures. I made an issue here to fix the creation of the RecordMapper. I made Issue #63 to handle this failure more gracefully.

@@ -20,8 +20,7 @@
<cranialDeformationCategory>urn:cspace:anthro.collectionspace.org:vocabularies:name(cranialdeformationcategory):item:name(tabular)'tabular'</cranialDeformationCategory>
<cranialDeformationCategory>urn:cspace:anthro.collectionspace.org:vocabularies:name(cranialdeformationcategory):item:name(circumferential)'circumferential'</cranialDeformationCategory>
</cranialDeformationCategories>
<Notes_NHTaphonomicAlterations>taphonomic alteration
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed new line since multiline field mapping worked ok in another test

@kevin-kariuki kevin-kariuki merged commit 5ca4cc8 into anthro-osteology Sep 11, 2020
@kspurgin kspurgin deleted the anthro-osteology-ks branch September 11, 2020 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants